Skip to content

Remove unused OperationContextBinding test wrappers#1903

Merged
vbabanin merged 2 commits intomongodb:mainfrom
vbabanin:JAVA-6118
Mar 20, 2026
Merged

Remove unused OperationContextBinding test wrappers#1903
vbabanin merged 2 commits intomongodb:mainfrom
vbabanin:JAVA-6118

Conversation

@vbabanin
Copy link
Member

@vbabanin vbabanin commented Mar 3, 2026

  • Delete OperationContextBinding and AsyncOperationContextBinding classes to eliminate code missed during CSOT refactoring
  • Remove unused ClusterFixture overloads (getBinding(TimeoutSettings), getBinding(OperationContext), getAsyncBinding(OperationContext), createNewOperationContext) to simplify the test API
  • Add applySessionContext to ReadOperation executeSync overloads to restore SessionContext application that was silently lost

JAVA-6118

- Delete OperationContextBinding and AsyncOperationContextBinding classes to eliminate code missed during CSOT refactoring
- Remove unused ClusterFixture overloads (getBinding(TimeoutSettings), getBinding(OperationContext), getAsyncBinding(OperationContext), createNewOperationContext) to simplify the test API
- Simplify binding factory methods to return ClusterBinding/AsyncClusterBinding directly instead of wrapping them in order to remove unnecessary indirection
- Add applySessionContext to ReadOperation executeSync overloads to restore SessionContext application that was silently lost

JAVA-6118
@vbabanin vbabanin self-assigned this Mar 3, 2026
@vbabanin vbabanin requested a review from rozza March 4, 2026 21:10
@vbabanin vbabanin marked this pull request as ready for review March 4, 2026 21:12
@vbabanin vbabanin requested a review from a team as a code owner March 4, 2026 21:12
Copy link
Member

@rozza rozza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@vbabanin vbabanin merged commit 2215413 into mongodb:main Mar 20, 2026
55 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants